Skip to content

Pin task version to 3.9.0 because of the PATH on win bug #677

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 30, 2021

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Nov 30, 2021

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • Tests for the changes have been added (for bug fixes / features)
  • What kind of change does this PR introduce?

Infrastructure fix

  • What is the current behavior?

The Test Go workflow on Windows is broken after the newest release of Task (3.9.1) and was working fine with 3.9.0
The problem should be linked to the upgrade in mvdan/sh they did in 3.9.1.
The bug is present only on Windows, apparently the PATH env variable is not expanded correctly under subcommands.

  • What is the new behavior?

To mitigate the issue, we decided to pin the task version used to 3.9.0 in the workflows

  • Does this PR introduce a breaking change?

no

  • Other information:

@umbynos umbynos added os: windows Specific to Windows operating system topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project labels Nov 30, 2021
@umbynos umbynos requested a review from per1234 November 30, 2021 16:47
@umbynos umbynos self-assigned this Nov 30, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #677 (7eab9c6) into main (714ffce) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #677   +/-   ##
=====================================
  Coverage   8.46%   8.46%           
=====================================
  Files         65      65           
  Lines       5046    5046           
=====================================
  Hits         427     427           
  Misses      4565    4565           
  Partials      54      54           
Flag Coverage Δ
unit 8.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ccfa7ae...7eab9c6. Read the comment docs.

@umbynos umbynos merged commit 320f7de into main Nov 30, 2021
@umbynos umbynos deleted the umbynos/pin_task branch November 30, 2021 17:16
umbynos added a commit that referenced this pull request Dec 3, 2021
umbynos added a commit that referenced this pull request Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
os: windows Specific to Windows operating system topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants